-
-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feat/giveaway blacklist #137
base: main
Are you sure you want to change the base?
Conversation
hilmoo
commented
Jul 21, 2024
•
edited
Loading
edited
- testing
this pull req extend #136 |
nvm, i forgot carl bot can handle that |
waiting for #136 to merge for testing |
bot/cogs/giveaway.py
Outdated
winner='Winner of the giveaway. Example: 1234567890,0987654321', | ||
ghosting='Member who do not claim the giveaway. Example: 1234567890,0987654321', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The example is too long for a parameter description
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think it's too long, it helps users understand how to separate them if more than one is entered
The code still has a syntax error. I’ll check it later, but for now, I can't test it on Discord. |
already tested the feature. lgtm |